Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

London 10 Maksim Lukianenko JavaScript-Core-2-Coursework-Week1 - #207

Open
maxbmaapc wants to merge 3 commits into
CodeYourFuture:mainfrom
maxbmaapc:main
Open

London 10 Maksim Lukianenko JavaScript-Core-2-Coursework-Week1#207
maxbmaapc wants to merge 3 commits into
CodeYourFuture:mainfrom
maxbmaapc:main

Conversation

@maxbmaapc

Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

Comment thread 2-mandatory/1-recipes.js
}
console.log(recipe1)
console.log(recipe2)
console.log(recipe3)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: well done

currencyCode[item[0]] = item[1];
}
return currencyCode;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: Well done!

Consider: You could also use .forEach https://www.w3schools.com/jsref/jsref_foreach.asp
function createLookup(countryCurrencyCodes) {
// write code here
const lookup = {};
countryCurrencyCodes.forEach(([country, currency]) => {
lookup[country] = currency;
});
return lookup;
}

const lookup = createLookup(COUNTRY_CURRENCY_CODES);
console.log(lookup);

name: recipe.name,
items: missingIngredients,
};
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: Well done!

orderFalafel: function (balance) {
return balance >= MENU.falafel? balance - MENU.falafel : balance;
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: Well done!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants